take the left, right, top and bottom attach arguments as mandatory and

Sat Oct 10 03:37:50 1998  Tim Janik  <timj@gtk.org>

        * gtk/gtktable.c (gtk_table_set_child_arg): take the left, right, top
        and bottom attach arguments as mandatory and readjust the table size
        and opposing edge accordingly if required. this way the order in which
        the arguments are set doesn't matter.

diff --git a/ChangeLog b/ChangeLog
index faff0e90952c8e45f40929157e80b538730c80df..a6336ee93c2cac40a17d27533b4fd41467fa1b23 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+Sat Oct 10 03:37:50 1998  Tim Janik  <timj@gtk.org>
+
+       * gtk/gtktable.c (gtk_table_set_child_arg): take the left, right, top
+       and bottom attach arguments as mandatory and readjust the table size
+       and opposing edge accordingly if required. this way the order in which
+       the arguments are set doesn't matter.
+
 Thu Oct  8 15:53:59 1998  Owen Taylor  <otaylor@redhat.com>
 
        * gtk/gtktypeutils.c (gtk_type_class_init): Fixed warning

diff --git a/ChangeLog.pre-2-0 b/ChangeLog.pre-2-0
index faff0e90952c8e45f40929157e80b538730c80df..a6336ee93c2cac40a17d27533b4fd41467fa1b23 100644 (file)
--- a/ChangeLog.pre-2-0
+++ b/ChangeLog.pre-2-0
@@ -1,3 +1,10 @@
+Sat Oct 10 03:37:50 1998  Tim Janik  <timj@gtk.org>
+
+       * gtk/gtktable.c (gtk_table_set_child_arg): take the left, right, top
+       and bottom attach arguments as mandatory and readjust the table size
+       and opposing edge accordingly if required. this way the order in which
+       the arguments are set doesn't matter.
+
 Thu Oct  8 15:53:59 1998  Owen Taylor  <otaylor@redhat.com>
 
        * gtk/gtktypeutils.c (gtk_type_class_init): Fixed warning

diff --git a/ChangeLog.pre-2-10 b/ChangeLog.pre-2-10
index faff0e90952c8e45f40929157e80b538730c80df..a6336ee93c2cac40a17d27533b4fd41467fa1b23 100644 (file)
--- a/ChangeLog.pre-2-10
+++ b/ChangeLog.pre-2-10
@@ -1,3 +1,10 @@
+Sat Oct 10 03:37:50 1998  Tim Janik  <timj@gtk.org>
+
+       * gtk/gtktable.c (gtk_table_set_child_arg): take the left, right, top
+       and bottom attach arguments as mandatory and readjust the table size
+       and opposing edge accordingly if required. this way the order in which
+       the arguments are set doesn't matter.
+
 Thu Oct  8 15:53:59 1998  Owen Taylor  <otaylor@redhat.com>
 
        * gtk/gtktypeutils.c (gtk_type_class_init): Fixed warning

diff --git a/ChangeLog.pre-2-2 b/ChangeLog.pre-2-2
index faff0e90952c8e45f40929157e80b538730c80df..a6336ee93c2cac40a17d27533b4fd41467fa1b23 100644 (file)
--- a/ChangeLog.pre-2-2
+++ b/ChangeLog.pre-2-2
@@ -1,3 +1,10 @@
+Sat Oct 10 03:37:50 1998  Tim Janik  <timj@gtk.org>
+
+       * gtk/gtktable.c (gtk_table_set_child_arg): take the left, right, top
+       and bottom attach arguments as mandatory and readjust the table size
+       and opposing edge accordingly if required. this way the order in which
+       the arguments are set doesn't matter.
+
 Thu Oct  8 15:53:59 1998  Owen Taylor  <otaylor@redhat.com>
 
        * gtk/gtktypeutils.c (gtk_type_class_init): Fixed warning

diff --git a/ChangeLog.pre-2-4 b/ChangeLog.pre-2-4
index faff0e90952c8e45f40929157e80b538730c80df..a6336ee93c2cac40a17d27533b4fd41467fa1b23 100644 (file)
--- a/ChangeLog.pre-2-4
+++ b/ChangeLog.pre-2-4
@@ -1,3 +1,10 @@
+Sat Oct 10 03:37:50 1998  Tim Janik  <timj@gtk.org>
+
+       * gtk/gtktable.c (gtk_table_set_child_arg): take the left, right, top
+       and bottom attach arguments as mandatory and readjust the table size
+       and opposing edge accordingly if required. this way the order in which
+       the arguments are set doesn't matter.
+
 Thu Oct  8 15:53:59 1998  Owen Taylor  <otaylor@redhat.com>
 
        * gtk/gtktypeutils.c (gtk_type_class_init): Fixed warning

diff --git a/ChangeLog.pre-2-6 b/ChangeLog.pre-2-6
index faff0e90952c8e45f40929157e80b538730c80df..a6336ee93c2cac40a17d27533b4fd41467fa1b23 100644 (file)
--- a/ChangeLog.pre-2-6
+++ b/ChangeLog.pre-2-6
@@ -1,3 +1,10 @@
+Sat Oct 10 03:37:50 1998  Tim Janik  <timj@gtk.org>
+
+       * gtk/gtktable.c (gtk_table_set_child_arg): take the left, right, top
+       and bottom attach arguments as mandatory and readjust the table size
+       and opposing edge accordingly if required. this way the order in which
+       the arguments are set doesn't matter.
+
 Thu Oct  8 15:53:59 1998  Owen Taylor  <otaylor@redhat.com>
 
        * gtk/gtktypeutils.c (gtk_type_class_init): Fixed warning

diff --git a/ChangeLog.pre-2-8 b/ChangeLog.pre-2-8
index faff0e90952c8e45f40929157e80b538730c80df..a6336ee93c2cac40a17d27533b4fd41467fa1b23 100644 (file)
--- a/ChangeLog.pre-2-8
+++ b/ChangeLog.pre-2-8
@@ -1,3 +1,10 @@
+Sat Oct 10 03:37:50 1998  Tim Janik  <timj@gtk.org>
+
+       * gtk/gtktable.c (gtk_table_set_child_arg): take the left, right, top
+       and bottom attach arguments as mandatory and readjust the table size
+       and opposing edge accordingly if required. this way the order in which
+       the arguments are set doesn't matter.
+
 Thu Oct  8 15:53:59 1998  Owen Taylor  <otaylor@redhat.com>
 
        * gtk/gtktypeutils.c (gtk_type_class_init): Fixed warning

diff --git a/TODO b/TODO
index 53589a84234de133ed497e269e29a903f095c34c..563a52c89a3b23bc761929ea1d6bb0a26dc332a8 100644 (file)
--- a/TODO
+++ b/TODO
@@ -6,7 +6,19 @@ Bugs:
  * Widget redrawing when the window resizes sometimes messes up.
    GtkLabels sometimes redraw without clearing up the underlying background on
    window resizes.
+
+ * Change bitfields to guints from enums, or vice versa?
+
+ * MappingNotify events produce warnings.
+
+ * gtk_widget_set_uposition can't handle negative x,y coordinates!
+ 
+ * the type system (gtktypeutils.c) has to handle creations of fundamental
+   types seperatedly from derived types, so we don't screw foreign
+   fundamental types with an already extensively increased seqno.
  
+ * Expose events aren't being generated correctly for DND demo
+
  * delay dnd settings to take effect once a widget is realized, this is
    to avoid force realizations. i think this goes along with owens dnd
    changes?
@@ -35,14 +47,6 @@ Bugs:
    segfault in malloc
     -timj
 
- * Change bitfields to guints from enums, or vice versa?
-
- * Expose events aren't being generated correctly for DND demo
-
- * MappingNotify events produce warnings.
-
- * gtk_widget_set_uposition can't handle negative x,y coordinates!
-
 Additions:
  * implement keyboard navigation in menus
 

diff --git a/gtk/gtktable.c b/gtk/gtktable.c
index 08e048f26673f1cd487369ff5fbe8a33a75247ec..ad09b98bc215c830dc8d4275fcb4072dd0342a69 100644 (file)
--- a/gtk/gtktable.c
+++ b/gtk/gtktable.c
@@ -258,25 +258,39 @@ gtk_table_set_child_arg (GtkContainer   *container,
   switch (arg_id)
     {
     case CHILD_ARG_LEFT_ATTACH:
-      if (GTK_VALUE_UINT (*arg) < table_child->right_attach)
-       table_child->left_attach = GTK_VALUE_UINT (*arg);
-      break;
-    case CHILD_ARG_RIGHT_ATTACH:
-      if (GTK_VALUE_UINT (*arg) > table_child->left_attach)
-       table_child->right_attach = GTK_VALUE_UINT (*arg);
+      table_child->left_attach = GTK_VALUE_UINT (*arg);
+      if (table_child->right_attach <= table_child->left_attach)
+       table_child->right_attach = table_child->left_attach + 1;
       if (table_child->right_attach >= table->ncols)
        gtk_table_resize (table, table->ncols, table_child->right_attach);
       break;
-    case CHILD_ARG_TOP_ATTACH:
-      if (GTK_VALUE_UINT (*arg) < table_child->bottom_attach)
-       table_child->top_attach = GTK_VALUE_UINT (*arg);
+    case CHILD_ARG_RIGHT_ATTACH:
+      if (GTK_VALUE_UINT (*arg) > 0)
+       {
+         table_child->right_attach = GTK_VALUE_UINT (*arg);
+         if (table_child->right_attach <= table_child->left_attach)
+           table_child->left_attach = table_child->right_attach - 1;
+         if (table_child->right_attach >= table->ncols)
+           gtk_table_resize (table, table->ncols, table_child->right_attach);
+       }
       break;
-    case CHILD_ARG_BOTTOM_ATTACH:
-      if (GTK_VALUE_UINT (*arg) > table_child->top_attach)
-       table_child->bottom_attach = GTK_VALUE_UINT (*arg);
+    case CHILD_ARG_TOP_ATTACH:
+      table_child->top_attach = GTK_VALUE_UINT (*arg);
+      if (table_child->bottom_attach <= table_child->top_attach)
+       table_child->bottom_attach = table_child->top_attach + 1;
       if (table_child->bottom_attach >= table->nrows)
        gtk_table_resize (table, table_child->bottom_attach, table->ncols);
       break;
+    case CHILD_ARG_BOTTOM_ATTACH:
+      if (GTK_VALUE_UINT (*arg) > 0)
+       {
+         table_child->bottom_attach = GTK_VALUE_UINT (*arg);
+         if (table_child->bottom_attach <= table_child->top_attach)
+           table_child->top_attach = table_child->bottom_attach - 1;
+         if (table_child->bottom_attach >= table->nrows)
+           gtk_table_resize (table, table_child->bottom_attach, table->ncols);
+       }
+      break;
     case CHILD_ARG_X_OPTIONS:
       table_child->xexpand = (GTK_VALUE_FLAGS (*arg) & GTK_EXPAND) != 0;
       table_child->xshrink = (GTK_VALUE_FLAGS (*arg) & GTK_SHRINK) != 0;